Add category name and simplify test names#57
Conversation
joehart2001
left a comment
joehart2001
left a comment
There was a problem hiding this comment.
Looks good, will give it a test. I'm also thinking we should add an extended description which is collapsable (think we've discussed briefly before). can add it to the yaml file
Thanks! And/or link to the documentation where we describe the category in detail (in addition to each test, as we have at the moment). |
yeah i think would be good to have both, as people may not want to open loads of links and just have a quick glance. will barely take up any space when colapsed and should be simple to add |
just tested and works as expected! Also, sorting the cols appears to update the linked interactive plot correctly too which is great and didn't realise worked yet |
ElliottKasoar
force-pushed
the
add-cat-details
branch
from
f44ce05 to
746362f
Compare
|
(Rebased) |
Pull Request Test Coverage Report for Build 17948956721Details
💛 - Coveralls |
4 participants
Pre-review checklist for PR author
PR author must check the checkboxes below when creating the PR.
Summary
Adds yaml files for category title, used for the tab, title, etc., and description.
Also combined individual test names/titles into a single name parameter, as we don't need it to be extracted from the file name, and this allows more human-readable text.
Linked issue
Resolves #43
Testing
None
Could use testing from @joehart2001, and improvements to the category descriptions are welcome!